Skip to content

fix:Refactor Collector component URLs to distribution/name structure#420

Open
omthorat2004 wants to merge 1 commit into
open-telemetry:mainfrom
omthorat2004:feat/collector-url-restructure
Open

fix:Refactor Collector component URLs to distribution/name structure#420
omthorat2004 wants to merge 1 commit into
open-telemetry:mainfrom
omthorat2004:feat/collector-url-restructure

Conversation

@omthorat2004
Copy link
Copy Markdown

fixes #368
This PR refactors Collector component routing and ID structure.

Changes

  • Component ID format changed from:
    {distribution}-{type}-{name}
    → {distribution}-{name}

  • Updated frontend routing:
    /collector/components/:version/:id
    → /collector/components/:distribution/:name

  • Version moved from path to query param:
    ?version=0.150.0

  • Updated data loading and transformation logic

  • Updated tests and assertions to match new ID format

Migration

Regenerated collector data with updated ID format. No manual migration required.

@omthorat2004 omthorat2004 requested review from a team as code owners May 10, 2026 13:53
@netlify
Copy link
Copy Markdown

netlify Bot commented May 10, 2026

Deploy Preview for otel-ecosystem-explorer ready!

Name Link
🔨 Latest commit 3db22fa
🔍 Latest deploy log https://app.netlify.com/projects/otel-ecosystem-explorer/deploys/6a0497a9a5c76f000820fe2c
😎 Deploy Preview https://deploy-preview-420--otel-ecosystem-explorer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@omthorat2004 omthorat2004 force-pushed the feat/collector-url-restructure branch from a123b1c to ad4be3f Compare May 10, 2026 13:55
@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented May 11, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: omthorat2004 / name: Om Thorat (c2b3157)

@vitorvasc
Copy link
Copy Markdown
Member

Hi @omthorat2004, could you sign the CLA?

@vitorvasc vitorvasc added add-screenshots missing:cla The CLA has not been signed by the PR author, therefore the PR can not be merged. and removed add-screenshots labels May 11, 2026
@omthorat2004
Copy link
Copy Markdown
Author

omthorat2004 commented May 11, 2026

@vitorvasc CLA is done!
Thanks.
I was also thinking to add add-screenshots label as these are ui changes.

@omthorat2004 omthorat2004 force-pushed the feat/collector-url-restructure branch 3 times, most recently from 1838b44 to c2b3157 Compare May 11, 2026 09:35
@jaydeluca
Copy link
Copy Markdown
Member

@omthorat2004 could you update your branch with main so the diff is clean?

I was also thinking to add add-screenshots label as these are ui changes.

Feel free to throw screenshots from testing into the PR description or comments, we always welcome that to help provide context on reviews 👍

@omthorat2004
Copy link
Copy Markdown
Author

@jaydeluca I will update my branch today.
Just need to change two files according to my changes.

@omthorat2004 omthorat2004 force-pushed the feat/collector-url-restructure branch 2 times, most recently from aa59e2b to 4afc06d Compare May 13, 2026 04:54
@omthorat2004
Copy link
Copy Markdown
Author

@jaydeluca check now.

@omthorat2004
Copy link
Copy Markdown
Author

omthorat2004 commented May 13, 2026

Collector Page

image

Collector Detail Page

image

@omthorat2004
Copy link
Copy Markdown
Author

omthorat2004 commented May 13, 2026

@vitorvasc Could you please remove "missing:cla" label.
I have already signed a CLA.
Thank you.

Hello @vitorvasc @jaydeluca could you please review this pr :).

@omthorat2004 omthorat2004 force-pushed the feat/collector-url-restructure branch from d6eb860 to 3db22fa Compare May 13, 2026 15:24
@omthorat2004
Copy link
Copy Markdown
Author

@jaydeluca I have also added screenshots of ui changes.
Also changed whole public/data/collector according to new format.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

missing:cla The CLA has not been signed by the PR author, therefore the PR can not be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Restructure Collector Component URLs

3 participants